Skip to content

#539 Remove message about builtins.#540

Merged
Matthew Hambley (MatthewHambley) merged 6 commits intomainfrom
539_remove_builtin_message
Apr 7, 2026
Merged

#539 Remove message about builtins.#540
Matthew Hambley (MatthewHambley) merged 6 commits intomainfrom
539_remove_builtin_message

Conversation

@hiker
Copy link
Copy Markdown
Collaborator

Tiny fix to remove unnecessary message in log files. Fixes #539.

@hiker Joerg Henrichs (hiker) added the Ready for review Indicating that a PR is ready to be reviewed. label Jan 22, 2026
@MatthewHambley
Copy link
Copy Markdown
Collaborator

See stfc/PSyclone#3291 for discussion of a proper API to obtain built-in names.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one question.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that BUILTIN_MAP is the official route to this information, we can use it.

@MatthewHambley Matthew Hambley (MatthewHambley) merged commit e341d7f into main Apr 7, 2026
8 checks passed
@MatthewHambley Matthew Hambley (MatthewHambley) deleted the 539_remove_builtin_message branch April 7, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready for review Indicating that a PR is ready to be reviewed.

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Verify builtins

2 participants